Conversation
There was a problem hiding this comment.
Pull request overview
This PR modernizes the Debian packaging by upgrading from debhelper version 7/9 to debhelper 13, following the new debhelper-compat build dependency pattern.
Changes:
- Updated Build-Depends to use debhelper-compat (= 13) instead of the old debhelper (>= 7)
- Removed the deprecated debian/compat file
- Added debian/not-installed file to document intentionally unpackaged files
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| debian/control | Updated Build-Depends to use debhelper-compat (= 13) format |
| debian/compat | Removed deprecated compat file (replaced by debhelper-compat in control) |
| debian/not-installed | Added file to document uninstalled man pages and libtool archives |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -0,0 +1,2 @@ | |||
| usr/share/man/man*/* | |||
There was a problem hiding this comment.
The pattern 'usr/share/man/man*/*' in the not-installed file will exclude ALL man pages from being installed. However, the package has multiple .manpages files (debian/libyaz5-dev.manpages, debian/yaz.manpages, debian/yaz-icu.manpages, debian/yaz-illclient.manpages) that explicitly list man pages to be included in specific packages. This creates a conflict where the man pages are being explicitly installed via .manpages files but then excluded via not-installed. Either remove this pattern from not-installed to allow the man pages to be installed as configured, or remove the .manpages files if man pages should not be included.
| usr/share/man/man*/* |
1 participant
No description provided.